Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat/script parser #385

Merged
merged 50 commits into from
Sep 7, 2024
Merged

Feat/script parser #385

merged 50 commits into from
Sep 7, 2024

Conversation

RLKRo
Copy link
Member

@RLKRo RLKRo commented Sep 7, 2024

Description

Added option to initialize Pipeline from yaml/json files.

Changelog

Breaking changes

  • Removed proc.ExtractAll -- the function is unsafe as it overwrites the entire slot storage. it is still available as method of the slot manager

Features

  • Add Pipeline from file import
  • Add function that creates an index of commonly-used Chatsky objects
  • Added imports to some __init__ files
  • Allow initializing NodeLabel from a list of two strings

Bug fixes

  • Slot extraction will now not write the value to the slot storage if value was not successfully extracted. Can be changed via the success_only flag

Devel

  • Add aliases to script keywords

Checklist

  • I have performed a self-review of the changes

To Consider

  • Add tests (if functionality is changed)
  • Update API reference / tutorials / guides
  • Update CONTRIBUTING.md (if devel workflow is changed)
  • Update .ignore files, scripts (such as lint), distribution manifest (if files are added/deleted)
  • Search for references to changed entities in the codebase

RLKRo and others added 30 commits February 12, 2024 02:05
@RLKRo
qol: add attachments to cli interface
481fd55
@RLKRo
qol: make keywords upper case
973abaa 
This allows using `Keywords.GLOBAL` and literal `"GLOBAL"` interchangeably.
@RLKRo
qol: accept list of attachments as Message.attachment
98692f9 
This commit will conflict with #328 and can be dropped.
@RLKRo
qol: make choice function accept responses as args
a9bbbb9
@RLKRo
qol: import script objects inside dff/__init__.py
d25fd30 
This allows importing `Message` directly from `dff`, among other import improvements.
@RLKRo
json import draft
e8fc8f6
@RLKRo
add json import examples
dfd61b5
@RLKRo
add submodule aliases
66b3bb0
@RLKRo
update resolve_target_object to use submodule aliases
9637110
@RLKRo
add function to retrieve dff object mapping
c4f04ba
@RLKRo
small fixes
cfda4f7
@RLKRo
capitalize messages
0e5f72a
@RLKRo
improve import system for custom objects
56755b9
@RLKRo
make custom_dir standard way to address custom dir
4b4cf69
@RLKRo
prepare parser for dev release
ce18579
@Ramimashkouk
Merge branch 'master' into 'chore/slots2parser'
702ddc8
@Ramimashkouk
Merge branch 'master' into chore/slots2parser
0c53232
@Ramimashkouk
feat: Integrate slots parsing
d787906
@Ramimashkouk
chore: Add pre-transitions to script example
8a4781d
@Ramimashkouk
refactor: Use a consistent approach with slots
486a432
@RLKRo
fix annotated import
8ff6bf0
@RLKRo
remove redundant config parsing
6174f9a
@RLKRo
fix slot regex in examples
acc2bc0
@RLKRo
feat: Add slots to parser (#375)
0c8f0a0 
The master was merged into script-parser, and a slot parsing
functionality was added.
@RLKRo
remove redundant imports
c697176
@RLKRo
update project version
763cb09
@RLKRo
Merge branch 'refs/heads/dev' into feat/script-parser
ef8b643 
# Conflicts:
#	chatsky/core/service/script_parsing.py
#	chatsky/pipeline/pipeline/pipeline.py
#	chatsky/script/core/keywords.py
#	chatsky/script/core/script.py
#	chatsky/script/responses/std_responses.py
#	chatsky/slots/slots.py
@RLKRo
allow initializing node label from lists and add tests
5fc1fcf
@RLKRo
allow capitalizing script keywords
43e4bf9
@RLKRo
fix merge conflict
9f8e73f
RLKRo added 17 commits September 6, 2024 20:50
@RLKRo
remove unnecessary imports; add more imports for modules
6857649 
This also updates telegram tutorial to mention another way to import the interface.
@RLKRo
rewrite json importer & add tests
aaee9d7
@RLKRo
allow changing NAMESPACE_PREFIX vars; simplify import methods
45c7fad
@RLKRo
allow importing from any library
e42ed3c
@RLKRo
add tests for get_chatsky_objects
c2d054a
@RLKRo
add api ref
baca438
@RLKRo
fix parse_args
5832aa4
@RLKRo
make slot extraction to not store unsuccessful values by default
8572c62
@RLKRo
remove ExtractAll processing
b8bbd1d 
It doesn't make it clear that it will overwrite all slots even if they are not successfully extracted.
SlotManager.extract_all is still available.
@RLKRo
update yaml import example; remove json import example
cef280d
@RLKRo
lint
07f92e5
@RLKRo
minor fixes for user_guides.rst
0e93212
@RLKRo
revert version change
bc1e740
@RLKRo
make yaml optional dependency
daf6451
@RLKRo
add user guide
a1b3005
@RLKRo
revert icon change
5429da9
@RLKRo
update poetry lock
68b2a4d
@RLKRo RLKRo added the enhancement New feature or request label Sep 7, 2024
@RLKRo RLKRo self-assigned this Sep 7, 2024
@RLKRo RLKRo merged commit 40981ef into dev Sep 7, 2024
17 checks passed
@RLKRo RLKRo mentioned this pull request Sep 7, 2024
Merged
@RLKRo RLKRo mentioned this pull request Sep 27, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@RLKRo RLKRo

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RLKRo @Ramimashkouk